Skip to content

apply clock layer range for nangate45 and sky130hs#3174

Merged
eder-matheus merged 8 commits into
The-OpenROAD-Project:masterfrom
eder-matheus:clock_layer_range
Jun 5, 2025
Merged

apply clock layer range for nangate45 and sky130hs#3174
eder-matheus merged 8 commits into
The-OpenROAD-Project:masterfrom
eder-matheus:clock_layer_range

Conversation

@eder-matheus

@eder-matheus eder-matheus commented May 19, 2025

Copy link
Copy Markdown
Member

designs/nangate45/gcd/rules-base.json updates:

Metric Old New Type
detailedroute__route__wirelength 5558 6853 Failing
finish__timing__setup__ws -0.07 -0.09 Failing
finish__design__instance__area 909 1069 Failing

designs/nangate45/aes/rules-base.json updates:

Metric Old New Type
finish__timing__setup__ws -0.14 -0.11 Tighten
finish__timing__wns_percent_delay -21.31 -18.58 Tighten

designs/nangate45/dynamic_node/rules-base.json updates:

Metric Old New Type
synth__design__instance__area__stdcell 25608.42 25515.12 Tighten
placeopt__design__instance__count__stdcell 13598 13523 Tighten
cts__design__instance__count__setup_buffer 1182 1176 Tighten
cts__design__instance__count__hold_buffer 1182 1176 Tighten
finish__timing__setup__ws -0.51 -0.49 Tighten
finish__timing__drv__setup_violation_count 591 588 Tighten
finish__timing__wns_percent_delay -33.08 -31.32 Tighten

designs/nangate45/ibex/rules-base.json updates:

Metric Old New Type
finish__timing__setup__ws -0.26 -0.19 Tighten
finish__timing__drv__setup_violation_count 832 825 Tighten
finish__timing__wns_percent_delay -18.58 -15.31 Tighten

designs/nangate45/jpeg/rules-base.json updates:

Metric Old New Type
synth__design__instance__area__stdcell 103107.27 103045.48 Tighten

designs/nangate45/swerv/rules-base.json updates:

Metric Old New Type
finish__timing__drv__hold_violation_count 408 185 Tighten

designs/nangate45/swerv_wrapper/rules-base.json updates:

Metric Old New Type
detailedroute__route__wirelength 5922536 5508164 Tighten
finish__timing__setup__ws -0.63 -0.52 Tighten

designs/nangate45/tinyRocket/rules-base.json updates:

Metric Old New Type
finish__timing__setup__ws -0.31 -0.21 Tighten
finish__timing__wns_percent_delay -27.52 -20.85 Tighten

designs/nangate45/bp_fe_top/rules-base.json updates:

Metric Old New Type
finish__timing__drv__hold_violation_count 1931 860 Tighten

designs/nangate45/bp_be_top/rules-base.json updates:

Metric Old New Type
finish__timing__setup__ws -0.76 -0.51 Tighten
finish__timing__wns_percent_delay -33.25 -29.19 Tighten

designs/nangate45/bp_multi_top/rules-base.json updates:

Metric Old New Type
detailedroute__route__wirelength 4890576 4806328 Tighten
finish__timing__setup__ws -4.36 -4.29 Tighten
finish__timing__drv__hold_violation_count 1026 100 Tighten

designs/nangate45/ariane133/rules-base.json updates:

Metric Old New Type
finish__timing__wns_percent_delay -11.66 -11.6 Tighten

designs/nangate45/ariane136/rules-base.json updates:

Metric Old New Type
finish__timing__drv__hold_violation_count 509 178 Tighten

designs/nangate45/black_parrot/rules-base.json updates:

Metric Old New Type
synth__design__instance__area__stdcell 778508.16 778458.91 Tighten
placeopt__design__instance__count__stdcell 299688 299499 Tighten
cts__design__instance__count__setup_buffer 26060 26043 Tighten
cts__design__instance__count__hold_buffer 26060 26043 Tighten
detailedroute__route__wirelength 10420028 8591633 Tighten
finish__timing__drv__setup_violation_count 13030 13022 Tighten
finish__timing__drv__hold_violation_count 429 100 Tighten

designs/sky130hs/aes/rules-base.json updates:

Metric Old New Type
placeopt__design__instance__area 177040 176428 Tighten
globalroute__antenna_diodes_count 53 9 Tighten
detailedroute__antenna_diodes_count 45 24 Tighten
finish__timing__setup__ws -0.47 -0.18 Tighten

designs/sky130hs/jpeg/rules-base.json updates:

Metric Old New Type
synth__design__instance__area__stdcell 654669.88 653350.08 Tighten
placeopt__design__instance__area 750737 749317 Tighten
globalroute__antenna_diodes_count 910 712 Tighten
detailedroute__antenna__violating__nets 3 1 Tighten
detailedroute__antenna_diodes_count 810 699 Tighten

designs/sky130hs/riscv32i/rules-base.json updates:

Metric Old New Type
detailedroute__route__wirelength 411975 378010 Tighten
detailedroute__antenna_diodes_count 12 9 Tighten
finish__timing__setup__ws -0.32 -0.21 Tighten
finish__design__instance__area 149394 141659 Tighten

Signed-off-by: Eder Monteiro <emrmonteiro@precisioninno.com>
Signed-off-by: Eder Monteiro <emrmonteiro@precisioninno.com>
@eder-matheus eder-matheus marked this pull request as draft May 19, 2025 20:56
@eder-matheus eder-matheus added the UpdateRules Starts GHA to update rules label May 20, 2025
@openroad-ci openroad-ci removed the UpdateRules Starts GHA to update rules label May 20, 2025
Signed-off-by: Eder Monteiro <emrmonteiro@precisioninno.com>
Signed-off-by: Eder Monteiro <emrmonteiro@precisioninno.com>
@eder-matheus eder-matheus added the UpdateRules Starts GHA to update rules label May 26, 2025
@openroad-ci openroad-ci removed the UpdateRules Starts GHA to update rules label May 26, 2025
Signed-off-by: Eder Monteiro <emrmonteiro@precisioninno.com>
Signed-off-by: Eder Monteiro <emrmonteiro@precisioninno.com>
Signed-off-by: Eder Monteiro <emrmonteiro@precisioninno.com>
@eder-matheus eder-matheus marked this pull request as ready for review June 4, 2025 21:48
@eder-matheus eder-matheus requested a review from maliberty June 4, 2025 21:48
@maliberty

Copy link
Copy Markdown
Member

Are there gains as well as the metrics update in this PR?

@eder-matheus

Copy link
Copy Markdown
Member Author

Are there gains as well as the metrics update in this PR?

The dashboard reports a bunch of improvements, but I'm not sure how much we can trust in this report:
image

I can download the artifacts for the affected designs and update their metrics locally. Then I can add the table with the updated metrics here, before merging the PR.

Signed-off-by: Eder Monteiro <emrmonteiro@precisioninno.com>
@eder-matheus

Copy link
Copy Markdown
Member Author

@maliberty I've added the metric updates for the remaining designs. We have a bunch of improvements, but some of them are not related to the layer range change.

@eder-matheus eder-matheus enabled auto-merge June 5, 2025 00:31
@eder-matheus eder-matheus merged commit d5f6f3a into The-OpenROAD-Project:master Jun 5, 2025
7 checks passed
@eder-matheus eder-matheus deleted the clock_layer_range branch June 5, 2025 14:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants